Skip to content

[18.0][IMP] sale_elaboration: auto notes parameter#4228

Merged
OCA-git-bot merged 1 commit intoOCA:18.0from
moduon:18.0-imp-sale_elaboration-notes
Apr 13, 2026
Merged

[18.0][IMP] sale_elaboration: auto notes parameter#4228
OCA-git-bot merged 1 commit intoOCA:18.0from
moduon:18.0-imp-sale_elaboration-notes

Conversation

@chienandalu
Copy link
Copy Markdown
Member

For some workflow cases it's annoying to have the elaboration notes auto filled with elaboration title.

This commit adds a parameter to disable that behavior so everyone is happy with their own preference.

cc @moduon MT-14286

please review @Shide @yajo

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @sergio-teruel, @rafaelbn, @yajo, @CarlosRoca13,
some modules you are maintaining are being modified, check this out!

Copy link
Copy Markdown
Member

@yajo yajo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A small suggestion to improve predictability and usability.

@chienandalu chienandalu force-pushed the 18.0-imp-sale_elaboration-notes branch from bf80541 to 5c6ebb1 Compare March 31, 2026 13:24
For some workflow cases it's annoying to have the elaboration
notes auto filled with elaboration title.

This commit adds a parameter to disable that behavior so everyone
is happy with their own preference.

MT-14286
@chienandalu chienandalu force-pushed the 18.0-imp-sale_elaboration-notes branch from 5c6ebb1 to 092de39 Compare March 31, 2026 13:34
@chienandalu chienandalu requested a review from yajo March 31, 2026 13:34
Copy link
Copy Markdown
Member

@yajo yajo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge minor

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-4228-by-yajo-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 78de239 into OCA:18.0 Apr 13, 2026
8 of 9 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 15cf022. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants